Skip to content

Cape Town | 26-ITP-Jan | Isaac Abodunrin | Sprint 2 | Form Controls#1062

Open
bytesandroses wants to merge 16 commits intoCodeYourFuture:mainfrom
bytesandroses:feat/form-controls
Open

Cape Town | 26-ITP-Jan | Isaac Abodunrin | Sprint 2 | Form Controls#1062
bytesandroses wants to merge 16 commits intoCodeYourFuture:mainfrom
bytesandroses:feat/form-controls

Conversation

@bytesandroses
Copy link
Member

@bytesandroses bytesandroses commented Jan 25, 2026 •
edited
Loading

Checklist

  • I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
  • My changes meet the requirements of the task
  • I have tested my changes
  • My changes follow the style guide

Changelist

Wrote a form to gather customer details and order specifications, which included:

  • Customer name (defined as a text string of at least 2 characters)
  • Valid email address
  • 3 colors options for T-shirt
  • 6 size options for T-shirt

Additional spec requirements were also met, including:

  • Lighthouse accessibility score of 100
  • Semantic HTML for form
  • No CSS or JavaScript

@netlify
Copy link

netlify bot commented Jan 25, 2026 •
edited
Loading

✅ Deploy Preview for cyf-onboarding-module ready!

Name Link
🔨 Latest commit 23249ec
🔍 Latest deploy log https://app.netlify.com/projects/cyf-onboarding-module/deploys/69769fb2b88d7b0008afaa87
😎 Deploy Preview https://deploy-preview-1062--cyf-onboarding-module.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse		 2 paths audited
Performance: 100 (no change from production)
Accessibility: 100 (no change from production)
Best Practices: 100 (no change from production)
SEO: 86 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions

This comment has been minimized.

Sign in to view
@bytesandroses bytesandroses marked this pull request as draft January 25, 2026 22:53
@bytesandroses bytesandroses marked this pull request as ready for review January 25, 2026 22:53
@bytesandroses bytesandroses added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Jan 25, 2026
@github-actions

This comment has been minimized.

Sign in to view
@github-actions github-actions bot removed the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Jan 25, 2026
@bytesandroses bytesandroses added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Jan 25, 2026
@bytesandroses bytesandroses mentioned this pull request Jan 28, 2026
Closed
@Waldo-Strydom
Copy link

Waldo-Strydom commented Jan 31, 2026

Well done.

@Waldo-Strydom Waldo-Strydom added Complete Volunteer to add when work is complete and all review comments have been addressed. and removed Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. labels Jan 31, 2026
@bytesandroses bytesandroses mentioned this pull request Jan 31, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Complete Volunteer to add when work is complete and all review comments have been addressed.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bytesandroses @Waldo-Strydom